Skip to content

Conversation

@jeremylt
Copy link
Member

@jeremylt jeremylt commented Nov 7, 2025

Closes #1902

This makes the output inline with other logging for Ratel

@jeremylt
Copy link
Member Author

jeremylt commented Nov 7, 2025

MacOS jobs are failing for some reason unrelated to this PR

@jeremylt
Copy link
Member Author

jeremylt commented Nov 7, 2025

TypeError: pathlib.Path | None is not callable is at the heart of the issue with MacOS

@jeremylt
Copy link
Member Author

jeremylt commented Nov 7, 2025

    parser.add_argument('-o', '--output', type=Optional[Path], default=None, help='Output file to write test')

This is the offending line.

@jeremylt
Copy link
Member Author

Ok @zatkins-dev this is about to go in so you will be able to use it in Ratel once I update the image for CI

@jeremylt jeremylt requested a review from zatkins-dev November 11, 2025 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Ceed*SetNumViewTabs for all libCEED objects

2 participants